Skip to content

Refactoriza la creación de nuevos clientes en NewServicioAT - #85

Merged
daniel89fg merged 1 commit into
masterfrom
fix-comprobacion-duplicado-cliente
Sep 8, 2026
Merged

Refactoriza la creación de nuevos clientes en NewServicioAT#85
daniel89fg merged 1 commit into
masterfrom
fix-comprobacion-duplicado-cliente

Conversation

@EstebanSM85

Copy link
Copy Markdown
Contributor

https://facturascripts.com/roadmap/4989
Hace la comprobación como la hace el core actualmente.

  • Se eliminó la función checkDuplicateCustomer y se implementó la lógica de verificación de CIF/NIF duplicados directamente en saveNewCustomerAction.
  • Se añadió un nuevo método findCustomersByCifnif para buscar clientes existentes por CIF/NIF.
  • Se mejoró la gestión de errores y se implementó el manejo de transacciones al guardar nuevos clientes.
  • Se actualizó el formulario en NewServicioAT.html.twig para incluir un campo oculto que confirma el CIF/NIF.
  • Se creó un nuevo archivo de pruebas NewServicioAtTest.php para validar la creación de nuevos clientes y la detección de CIF/NIF duplicados.

https://facturascripts.com/roadmap/4989

- Se eliminó la función checkDuplicateCustomer y se implementó la lógica de verificación de CIF/NIF duplicados directamente en saveNewCustomerAction.
- Se añadió un nuevo método findCustomersByCifnif para buscar clientes existentes por CIF/NIF.
- Se mejoró la gestión de errores y se implementó el manejo de transacciones al guardar nuevos clientes.
- Se actualizó el formulario en NewServicioAT.html.twig para incluir un campo oculto que confirma el CIF/NIF.
- Se creó un nuevo archivo de pruebas NewServicioAtTest.php para validar la creación de nuevos clientes y la detección de CIF/NIF duplicados.
@daniel89fg
daniel89fg merged commit 47ce9cf into master Sep 8, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants